Skip to content

refactor(aria/combobox): make input readonly when disabled#33187

Merged
tjshiu merged 1 commit intoangular:mainfrom
tjshiu:disabled-fix-no-typing
May 2, 2026
Merged

refactor(aria/combobox): make input readonly when disabled#33187
tjshiu merged 1 commit intoangular:mainfrom
tjshiu:disabled-fix-no-typing

Conversation

@tjshiu
Copy link
Copy Markdown
Contributor

@tjshiu tjshiu commented May 1, 2026

  • issues when we have simple-combobox and softDisabled
  • need the input or textarea element to still be readable without marking it disabled since it would block the focus on it (preventing soft disabled)

@pullapprove pullapprove Bot requested review from andrewseguin and wagnermaciel May 1, 2026 22:58
@tjshiu tjshiu requested review from adolgachev and ok7sai May 1, 2026 22:59
@tjshiu tjshiu added target: major This PR is targeted for the next major release dev-app preview When applied, previews of the dev-app are deployed to Firebase labels May 1, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

Deployed dev-app for 6db4781 to: https://ng-dev-previews-comp--pr-angular-components-33187-dev-aaaa5eus.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@tjshiu tjshiu added the action: merge The PR is ready for merge by the caretaker label May 2, 2026
@tjshiu tjshiu merged commit 306fe00 into angular:main May 2, 2026
32 of 34 checks passed
@tjshiu
Copy link
Copy Markdown
Contributor Author

tjshiu commented May 2, 2026

This PR was merged into the repository. The changes were merged into the following branches:

@tjshiu tjshiu deleted the disabled-fix-no-typing branch May 2, 2026 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: aria/combobox dev-app preview When applied, previews of the dev-app are deployed to Firebase target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants